Skip to content
This repository has been archived by the owner on Nov 28, 2018. It is now read-only.

For issue http://projects.puppetlabs.com/issues/15192 #41

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

haiwu
Copy link

@haiwu haiwu commented Jun 24, 2012

This is for issue http://projects.puppetlabs.com/issues/15192, to support both noop and no-noop modes.

@ripienaar
Copy link
Contributor

Thanks, we'll need to update the DDL too else you wouldnt be able to send those arguments in later versions

@haiwu
Copy link
Author

haiwu commented Jun 25, 2012

I am not sure why this is not working:

After uncomment-ing the following lines in file 'puppetd.ddl', every time when I run "mco puppetd runonce -I hostname", it would fail with message "puppetd failed to run: Input forcerun should be a string (MCollective::DDLValidationError)", even though input "forcerun" is marked as "optional" ..

input :forcerun,
    :prompt      => "Force puppet run",
    :description => "Should the puppet run happen immediately?",
    :type        => :string,
    :validation  => '^.+$',
    :optional    => true,
    :maxlength   => 5

@ripienaar
Copy link
Contributor

Should probably make that :boolean, anyway will have to see how you are calling it from the cli

@puppetcla
Copy link

Waiting for CLA signature by @haiwu

@haiwu - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppetlabs.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppetlabs.com/community/trivial_patch_exemption.html

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants